-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refine search result by customizing analyzer #4456
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4456 +/- ##
============================================
+ Coverage 61.11% 61.15% +0.04%
Complexity 2523 2523
============================================
Files 372 372
Lines 12942 12940 -2
Branches 929 928 -1
============================================
+ Hits 7909 7913 +4
+ Misses 4582 4576 -6
Partials 451 451
|
Hi @JohnNiang, 此 PR 该如何测试,是否会对搜索结果和匹配效果产生影响 |
Hi @LIlGG , Query 解析语法在这里可以看到。不建议用户手动输入查询表达式,更建议在 https://github.com/halo-sigs/plugin-search-widget 中集成。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: guqing, ruibaby The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind improvement
/area core
/milestone 2.9.x
What this PR does / why we need it:
cn.shenyanchao.ik-analyzer:ik-analyzer:9.0.0
due to no significant effect for searching result.Please be aware of that the default field to search has become to
content
instead oftitle
+excerpt
+content
. If someone wants to search title only, usetitle: halo
as query string. For more details, please refer to https://lucene.apache.org/core/9_5_0/queryparser/org/apache/lucene/queryparser/flexible/standard/StandardQueryParser.html.Which issue(s) this PR fixes:
Fixes #4455
Special notes for your reviewer:
Does this PR introduce a user-facing change?